Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Upgrade simple history package #153

Merged
merged 4 commits into from
Oct 3, 2023
Merged

Conversation

UsamaSadiq
Copy link
Member

Description

  • PR created under the issue Upgrade django-simple-history in all IDAs edx/edx-arch-experiments#462
  • Added common_constraints.txt to override global simple-history package constraint
  • Bumped the django-simple-history package version to the latest
  • Added new migrations caused by the new version of django-simple-history package.
  • Added new setting to avoid db_indexing in the new migrations.

Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
Makefile Outdated Show resolved Hide resolved
@macdiesel
Copy link
Contributor

@UsamaSadiq Is this a blocker for Django 4.2?

@UsamaSadiq
Copy link
Member Author

@UsamaSadiq Is this a blocker for Django 4.2?

Yes, this would need to be merged before upgrading to Django 4.2

@macdiesel macdiesel merged commit 1b075ab into main Oct 3, 2023
6 checks passed
@macdiesel macdiesel deleted the upgrade-simple-history-package branch October 3, 2023 19:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants